Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename bitstream_status_list module into bitstring_status_list. #573

Merged
merged 6 commits into from
Jul 8, 2024

Conversation

timothee-haudebourg
Copy link
Contributor

@timothee-haudebourg timothee-haudebourg commented Jul 5, 2024

This PR makes a few adjustment on the Bitstring Status List implementation, by first renaming the wrongly named bitstream_status_list module into bitstring_status_list. But also:

  • make the fields of BitstringStatusListEntry public and add a constructor,
  • simplify serialization of the type property using serde's tag attribute instead of a singleton type BitstringStatusListEntryType,
  • same for BitstringStatusListType; and
  • add constructors and methods for StatusPurpose.

@timothee-haudebourg timothee-haudebourg force-pushed the improve-bitstring-status-list-types branch from f12b04c to d91f404 Compare July 5, 2024 10:41
@timothee-haudebourg timothee-haudebourg changed the title Make BitstringStatusListEntry's fields public Rename bitstream_status_list module into bitstring_status_list. Jul 5, 2024
@timothee-haudebourg timothee-haudebourg merged commit 90d11cb into main Jul 8, 2024
4 checks passed
@timothee-haudebourg timothee-haudebourg deleted the improve-bitstring-status-list-types branch July 8, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants